x11: Move function call
authorBenjamin Otte <otte@redhat.com>
Sun, 6 Jun 2021 15:18:16 +0000 (17:18 +0200)
committerBenjamin Otte <otte@redhat.com>
Thu, 22 Jul 2021 14:06:05 +0000 (16:06 +0200)
The GLX visual selection is GLX specific, so it can be handled by the GLX
code.

There should be no reordering here, the call was just moved.

gdk/x11/gdkdisplay-x11.c
gdk/x11/gdkglcontext-glx.c
gdk/x11/gdkglcontext-x11.h

index 5eaf09ec6b76932750662eac32e8cd5260a9ef76..5bdbe2ffdad1b7820617cdea5dae055acc9e4a73 100644 (file)
@@ -1426,7 +1426,6 @@ gdk_x11_display_open (const char *display_name)
    * stereo and double buffering
    */
   gdk_x11_display_init_gl (display_x11);
-  gdk_x11_screen_update_visuals_for_glx (display_x11->screen);
 
   if (display_x11->screen->rgba_visual)
     {
index a6306c6ed1ae0c0dcfbb91d178597123381746c0..bc6d66a7e56dc136cf6f3d7542f7ac89dbf172b7 100644 (file)
@@ -1005,7 +1005,7 @@ save_cached_gl_visual (GdkDisplay *display, VisualID visual)
   gdk_x11_display_error_trap_pop_ignored (display);
 }
 
-void
+static void
 gdk_x11_screen_update_visuals_for_glx (GdkX11Screen *x11_screen)
 {
   GdkDisplay *display;
@@ -1019,9 +1019,6 @@ gdk_x11_screen_update_visuals_for_glx (GdkX11Screen *x11_screen)
   display_x11 = GDK_X11_DISPLAY (display);
   dpy = gdk_x11_display_get_xdisplay (display);
 
-  if (!display_x11->have_glx)
-    return;
-
   /* We save the default visuals as a property on the root window to avoid
      having to initialize GL each time, as it may not be used later. */
   rgba_visual_id = get_cached_gl_visual (display);
@@ -1322,5 +1319,7 @@ gdk_x11_screen_init_glx (GdkX11Screen *screen)
                      display_x11->has_glx_multisample ? "yes" : "no",
                      display_x11->has_glx_visual_rating ? "yes" : "no"));
 
+  gdk_x11_screen_update_visuals_for_glx (display_x11->screen);
+
   return TRUE;
 }
index 991c361cc53c619e749678e25352e4e0dc9f9c41..eaba65df2a5e8e2cfcadcf9435b1c1af3000d7fc 100644 (file)
@@ -75,7 +75,6 @@ gboolean        gdk_x11_display_make_gl_context_current (GdkDisplay    *display,
 typedef struct _GdkX11GLContextGLX      GdkX11GLContextGLX;
 
 gboolean                gdk_x11_screen_init_glx                 (GdkX11Screen  *screen);
-void                    gdk_x11_screen_update_visuals_for_glx   (GdkX11Screen  *screen);
 
 GType                   gdk_x11_gl_context_glx_get_type         (void) G_GNUC_CONST;
 GdkX11GLContext *       gdk_x11_gl_context_glx_new              (GdkSurface    *surface,